Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Using Goerli RPC Gateway for simulations #253

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Using Goerli RPC Gateway for simulations #253

wants to merge 7 commits into from

Conversation

lgahdl
Copy link
Contributor

@lgahdl lgahdl commented Dec 21, 2022

Finished the implementation of RPC Gateway usage on tenderlyHelper;

  • Created a variable called "tenderlyRpcUrl";
  • Created a new function "simulateTransactionRpc" and replaced "simulateTransaction" for it on "simulateMulticall";

OBS.:

  1. RpcUrl is for Goerli, I can adapt it for other networks if needed.
  2. The output is filtered for the call with the method "multicall", I can adapt for other methods, if needed.

@lgahdl
Using Goerli RPC Gateway for simulations
a844f35 
Finished the implementation of RPC Gateway usage on tenderlyHelper;
@lgahdl lgahdl requested review from timjrobinson, johngrantuk, brunoguerios and gmbronco and removed request for timjrobinson December 21, 2022 23:00
@lgahdl
Fixing ESLint requirements
bced08a
@lgahdl
Fixing 2 ESLint Warnings
cb56399 
Unused variable;
Missing function return type;
@lgahdl
Commenting tenderlyUrl with user and project;
ac77226 
leaving only "https://api.balancer.fi/tenderly" endpoint
@lgahdl
Adding try catches to find the error;
0a14f80
@lgahdl
Putting the old "tenderlyUrl" (with user and project parameters) back;
a1dd35d 
To see if it affects in the CI tests;
@lgahdl
Removing try catch from encode state call;
e464497 
changing the console.error to show error.response.data.error, instead of error full object.
@lgahdl lgahdl self-assigned this Mar 2, 2023
@lgahdl lgahdl added the enhancement New feature or request label Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johngrantuk johngrantuk Awaiting requested review from johngrantuk

@brunoguerios brunoguerios Awaiting requested review from brunoguerios

@gmbronco gmbronco Awaiting requested review from gmbronco

At least 1 approving review is required to merge this pull request.

Assignees

@lgahdl lgahdl

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lgahdl